Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add reverse proxy step to onboarding for feature flags #20906

Merged
merged 11 commits into from
Mar 19, 2024

Conversation

zlwaterfield
Copy link
Contributor

@zlwaterfield zlwaterfield commented Mar 13, 2024

Problem

A reverse proxy allows users to send events to PostHog Cloud using your own domain. This means that events are sent from their own domain and are less likely to be intercepted by tracking blockers.

Product like feature flags require a proxy in order to make sure the necessary requests succeed for all users.

Changes

We already have the documentation for how to set up a proxy. This PR is adding the a step to the onboarding flow to let the user know they should set one up.

Few notes:

  • Right now it's only on the feature flags onboarding.
  • It is optional (noted in the step title)
  • Users can skip, complete or invite a team member to set it up

Screenshot 2024-03-14 at 10 43 52 AM

Other items discussed while doing this:

  • We also plan to add this to the Quick Start steps
  • We are looking into a way to be able to know whether or not a user has a proxy set up within in the product
  • In the future we may do more in onboarding like embedding the code directly instead of linking to the docs

@zlwaterfield zlwaterfield self-assigned this Mar 13, 2024
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

Copy link
Contributor

github-actions bot commented Mar 13, 2024

Size Change: +52 B (0%)

Total Size: 821 kB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 821 kB +52 B (0%)

compressed-size-action

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@zlwaterfield zlwaterfield marked this pull request as ready for review March 13, 2024 17:06
Copy link
Member

@raquelmsmith raquelmsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@zlwaterfield zlwaterfield force-pushed the onboarding-proxy-step branch from 810b0ea to e724cd5 Compare March 18, 2024 21:31
@zlwaterfield
Copy link
Contributor Author

Here is the A/B test: https://us.posthog.com/project/2/experiments/24377.

@zlwaterfield zlwaterfield merged commit f9865a1 into master Mar 19, 2024
80 checks passed
@zlwaterfield zlwaterfield deleted the onboarding-proxy-step branch March 19, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants